Skip to content

Migrate to oxfmt#618

Merged
Andarist merged 4 commits into
mainfrom
oxfmt
May 15, 2026
Merged

Migrate to oxfmt#618
Andarist merged 4 commits into
mainfrom
oxfmt

Conversation

@bluwy
Copy link
Copy Markdown
Member

@bluwy bluwy commented May 15, 2026

Enables import sorting and trailingComma: "always" so matches the changesets repo setup.

NOTE: It also sorts the package.json

@bluwy bluwy requested review from Andarist and emmatown as code owners May 15, 2026 08:01
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 15, 2026 •
edited
Loading

⚠️ No Changeset found

Latest commit: 672d875

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 15, 2026 •
edited
Loading

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security		 Vulnerability Quality Maintenance License
Addedoxfmt@​0.48.0901008996100

View full report

bluwy
bluwy commented May 15, 2026
View reviewed changes
Comment thread package.json
Comment on lines -56 to -60
"pnpm": {
"overrides": {
"trim": "^0.0.3",
"y18n": "^4.0.1"
}
Copy link
Copy Markdown
Member Author

@bluwy bluwy May 15, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is completely removed as pnpm v11 no longer reads it, only works when configured in pnpm-workspace.yaml.

It doesn't seem like we configure that and everything's working, so I didn't port it to pnpm-workspace.yaml.

Andarist
Andarist previously approved these changes May 15, 2026
View reviewed changes
@Andarist
Copy link
Copy Markdown
Member

Andarist commented May 15, 2026

@bluwy this requires conflict resolution now

@Andarist Andarist enabled auto-merge May 15, 2026 11:48
@Andarist Andarist added this pull request to the merge queue May 15, 2026
Merged via the queue into main with commit 6c80731 May 15, 2026
7 checks passed
@Andarist Andarist deleted the oxfmt branch May 15, 2026 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Andarist Andarist Andarist approved these changes

@emmatown emmatown Awaiting requested review from emmatown emmatown is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bluwy @Andarist